SRE-3706: Add groovylint and markdownlint#5
Merged
Conversation
Contributor
Author
|
Tested as a submodule and got the expected results. |
JohnMalmberg
requested review from
grom72,
janekmi,
jzarzycki,
osalyk and
ryon-jensen
JohnMalmberg
force-pushed
the
sre-3706-groovylint
branch
from
68d029a to
84a3f9a
Compare
- Add changed-file detection and runner wiring for groovylint and markdownlint.
- Pin npm-groovy-lint and markdownlint-cli versions.
- Improve Ubuntu/WSL Node/npm install handling for linter CLI setup.
- Update linter docs and tool mapping.
Changed files/directories
- bin/run-linters.sh, checks/detect-linters.sh, checks/linter-common.sh:
add linter selection and dispatch wiring for Groovy/Markdown files.
- checks/install-linter-tools.sh:
add pinned npm CLI installs and Ubuntu/WSL Node/npm reliability handling.
- checks/linters/groovylint/:
add groovylint scripts and implicit-binding safety check.
- checks/linters/markdownlint/:
add markdownlint scripts.
- docs/linters.md, docs/usage.md, docs/maintenance.md:
update linter/tooling documentation and wording.
- docs/integration.md:
Fix minor linting issue.
Signed-off-by: John E. Malmberg <john.malmberg@hpe.com>
JohnMalmberg
force-pushed
the
sre-3706-groovylint
branch
from
84a3f9a to
9761338
Compare
ryon-jensen
approved these changes
Apr 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changed files/directories